Skip to content

Update @github/copilot to 1.0.49-6#1327

Merged
stephentoub merged 10 commits into
mainfrom
update-copilot-1.0.49-6
May 19, 2026
Merged

Update @github/copilot to 1.0.49-6#1327
stephentoub merged 10 commits into
mainfrom
update-copilot-1.0.49-6

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Automated update of @github/copilot to version 1.0.49-6.

Changes

  • Updated @github/copilot in nodejs/package.json and test/harness/package.json
  • Re-ran all code generators (scripts/codegen)
  • Formatted generated output

Next steps

When ready, click Ready for review to trigger CI checks.

Created by the Update @github/copilot Dependency workflow.

@stephentoub stephentoub marked this pull request as ready for review May 18, 2026 22:18
@stephentoub stephentoub requested a review from a team as a code owner May 18, 2026 22:18
Copilot AI review requested due to automatic review settings May 18, 2026 22:18
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Comment thread dotnet/src/Generated/SessionEvents.cs Fixed
@stephentoub stephentoub self-assigned this May 18, 2026
Comment thread dotnet/test/E2E/ModeHandlersE2ETests.cs Fixed
Comment thread dotnet/src/SessionFsProvider.cs Fixed
Comment thread dotnet/src/SessionFsProvider.cs Fixed
@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor Author

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generated by SDK Consistency Review Agent for issue #1327 · ● 1.1M

Comment thread rust/src/generated/api_types.rs
Comment thread rust/src/session_fs.rs
Comment thread dotnet/test/E2E/ModeHandlersE2ETests.cs Fixed
@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

@github-actions

This comment has been minimized.

Copy link
Copy Markdown
Contributor Author

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Generated by SDK Consistency Review Agent for issue #1327 · ● 2M

Comment thread rust/src/session_fs.rs Outdated
github-actions Bot and others added 6 commits May 18, 2026 23:25
- Updated nodejs and test harness dependencies
- Re-ran code generators
- Formatted generated code
Add SessionFS SQLite support across SDKs and fix generated type issues exposed by the updated Copilot schema.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Tighten C# SessionFS exception handling and make Python SessionFS adapter params type-check cleanly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Teach Rust codegen to prefer richer inline RPC parameter schemas when definitions omit wire fields, and forward the SQLite query session ID through the provider trait.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@stephentoub stephentoub force-pushed the update-copilot-1.0.49-6 branch from b05364e to 8ba82ae Compare May 19, 2026 03:34
@github-actions

This comment has been minimized.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions

This comment has been minimized.

Copilot AI added 3 commits May 19, 2026 04:12
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor Author

Cross-SDK Consistency Review ✅

This PR (Update @github/copilot to 1.0.49-6) is an automated codegen update that regenerates all SDK code from the same schema. All five SDKs (Node.js/TypeScript, Python, Go, .NET, Rust) are updated in lock-step and maintain strong cross-language consistency.

Changes reviewed

Feature Node.js Python Go .NET Rust
SessionFsProvider.sqliteQuery sqliteQuery() sqlite_query() SqliteQuery() SqliteQueryAsync() sqlite_query()
SessionFsProvider.sqliteExists sqliteExists() sqlite_exists() SqliteExists() SqliteExistsAsync() sqlite_exists()
SessionFsSetProviderCapabilities (generated type)
AutoModeSwitchResponse moved to generated session-events
ToolBinaryResult.type narrowed to "image" | "resource" ✅ (hand-authored) ✅ (generated) ✅ (generated) ✅ (generated) ✅ (generated)

Error-handling consistency

The SQLite adapter error-collapsing pattern is applied uniformly:

  • sqliteQuery errors → { columns: [], rows: [], rowsAffected: 0, error: <mapped error> }
  • sqliteExists errors → { exists: false }

This matches the existing pattern used by readFile, exists, etc. across all SDKs.

Note on Rust default implementations

Rust's SessionFsProvider trait provides default implementations for sqlite_query and sqlite_exists (returning FsError::Other("not supported")), while other SDKs require implementors to provide the methods (abstract/interface). This is idiomatic per-language behavior — Rust traits commonly use defaults to avoid breaking existing implementors, while TypeScript/Python/Go/C# signal required surface via interface/abstract mechanisms.

No consistency issues found. 🎉

Generated by SDK Consistency Review Agent for issue #1327 · ● 1.4M ·

@stephentoub stephentoub merged commit 0894c17 into main May 19, 2026
44 checks passed
@stephentoub stephentoub deleted the update-copilot-1.0.49-6 branch May 19, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants